[DO NOT MERGE] debug pypi workflow #241
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@webknjaz, current results from running the anything else you want me to check? |
|
But what about the subdirs? |
|
Minimum allowed coverage is Generated by 🐒 cobertura-action against 99de9f8 |
|
this run runs |
|
Ah, so /opt/conda/bin/python3: can't open file '/home/githubrunner/actions-runner/_work/_actions/pypa/gh-action-pypi-publish/release/v1/create-docker-action.py': [Errno 2] No such file or directorySo it seems to me like GitHub Runner does put the host path into the The mount point is configured as |
|
So it seems to me like this is a case of a bug in GitHub Runner. And my hunch is that it's probably not caused by it being a hosted runner. It's likely caused by the use of the container setting on the job. @sroet to verify this, drop the container from the job and try again. If that's the case, I don't think we'll be attempting to support it (you can probably hack on your container and stick symlinks into it, but that's out of the scope). Though, I also noticed that you're not following the recommendation of keeping the build invocation out of the job that has elevated OIDC privileges. Perhaps, if you just split it according to https://packaging.python.org/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/, the problem will disappear. |
That's not happening as dropping the container from a self-hosted runner will run the code on the actual host machine. Our host machines are not isolated/revertible enough to let random code run on it outside of a container.
I can try that, and if that doesn't work maybe push the publishing part to github-runners instead of self-hosted ones |
That's exactly what I meant. |
|
So, splitting out the job to run the upload on github runners instead of self-hosted ones seems to work (the failure is expected as this branch can't be valid for the trusted publishing setup we have on this project) |
Merged
|
I think all the necessary debugging has been done, so I will close this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.